Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Manual Backport 2.x] Add support for register data sources during the absence of local cluster #2144

Merged
merged 2 commits into from
Sep 9, 2024
Merged

[Manual Backport 2.x] Add support for register data sources during the absence of local cluster #2144

merged 2 commits into from
Sep 9, 2024

Conversation

RyanL1997
Copy link
Collaborator

Description

Add support for register data sources during the absence of local cluster

Issues Resolved

  • Resolve the 500 errors when there is no backend plugins to consume the requests

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@RyanL1997
Add support for rigister plugin during the absence of local cluster
c824975 
Signed-off-by: Ryan Liang <jiallian@amazon.com>
@RyanL1997 RyanL1997 added enhancement New feature or request backport 2.17 labels Sep 6, 2024
@RyanL1997
Update release note
0f8bf37 
Signed-off-by: Ryan Liang <jiallian@amazon.com>
@RyanL1997 RyanL1997 mentioned this pull request Sep 6, 2024
Merged
6 tasks
@codecov Codecov
Copy link

codecov bot commented Sep 6, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.48%. Comparing base (fc85856) to head (0f8bf37).
Report is 21 commits behind head on 2.x.

Additional details and impacted files 
@@            Coverage Diff             @@
##              2.x    #2144      +/-   ##
==========================================
- Coverage   57.01%   56.48%   -0.53%     
==========================================
  Files         379      390      +11     
  Lines       14541    14975     +434     
  Branches     3898     4066     +168     
==========================================
+ Hits         8290     8459     +169     
- Misses       6183     6454     +271     
+ Partials       68       62       -6
Flag Coverage Δ
dashboards-observability 56.48% <ø> (-0.53%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@RyanL1997 RyanL1997 merged commit 12baaea into opensearch-project:2.x Sep 9, 2024
9 of 19 checks passed
@RyanL1997 RyanL1997 deleted the neo-support-2x branch September 9, 2024 19:04
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 9, 2024
@github-actions
[Manual Backport 2.x] Add support for register data sources during th…
96bdcb2 
…e absence of local cluster (#2144)

(cherry picked from commit 12baaea)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Sep 9, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joshuali925 joshuali925 joshuali925 approved these changes

@TackAdam TackAdam TackAdam approved these changes

@ps48 ps48 Awaiting requested review from ps48 ps48 is a code owner

@kavithacm kavithacm Awaiting requested review from kavithacm kavithacm is a code owner

@derek-ho derek-ho Awaiting requested review from derek-ho derek-ho is a code owner

@dai-chen dai-chen Awaiting requested review from dai-chen dai-chen is a code owner

@YANG-DB YANG-DB Awaiting requested review from YANG-DB YANG-DB is a code owner

@rupal-bq rupal-bq Awaiting requested review from rupal-bq rupal-bq is a code owner

@mengweieric mengweieric Awaiting requested review from mengweieric mengweieric is a code owner

@vamsi-amazon vamsi-amazon Awaiting requested review from vamsi-amazon vamsi-amazon is a code owner

@Swiddis Swiddis Awaiting requested review from Swiddis Swiddis is a code owner

@penghuo penghuo Awaiting requested review from penghuo penghuo is a code owner

@seankao-az seankao-az Awaiting requested review from seankao-az seankao-az is a code owner

@anirudha anirudha Awaiting requested review from anirudha anirudha is a code owner

Assignees
No one assigned
Labels
backport 2.17 enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@RyanL1997 @joshuali925 @TackAdam